Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add automatic line ending conversion #96

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

zikato
Copy link

@zikato zikato commented Apr 1, 2023

https://docs.github.com/en/get-started/getting-started-with-git/configuring-git-to-handle-line-endings#per-repository-settings

Internally it's stored as LF, but it converts to the native line endings on checkout. So Windows users will have CRLF, and Linux will have LF.

AutoFormatSQL Action runs upon pushing to the master branch (after PR is merged).

  • Removes trailing whitespace
  • Replaces tabs with the correct amount of spaces (tab size = 4)
  • Removes blank lines at the end of the file

Requires: GitHub repo page - Settings\Actions\General\Workflow permissions\Read and write permissions
Thanks to permissions content:write no longer needs the token

zikato added 4 commits April 1, 2023 21:45
Removes trailing whitespace
Replaces tabs with a correct amount of spaces
Removes a blank line from end
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant